Skip to content

59 task write trails drawing feature functions calculate smoothness#94

Merged
cgmaiorano merged 12 commits into
mainfrom
59-task-write-trails-drawing-feature-functions-calculate_smoothness
Feb 9, 2026
Merged

59 task write trails drawing feature functions calculate smoothness#94
cgmaiorano merged 12 commits into
mainfrom
59-task-write-trails-drawing-feature-functions-calculate_smoothness

Conversation

@cgmaiorano
Copy link
Copy Markdown
Collaborator

@cgmaiorano cgmaiorano commented Dec 9, 2025

The calculate_smoothness function calculates the path smoothness based on curvature changes. Lower values indicate smoother paths. This function calculates angles between consecutive segments and returns the standard deviation of these angles as a measure of smoothness.

This function is calculated per segment

@cgmaiorano cgmaiorano linked an issue Dec 9, 2025 that may be closed by this pull request
4 tasks
@cgmaiorano cgmaiorano marked this pull request as ready for review December 9, 2025 15:50
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.55%. Comparing base (704fce8) to head (6045ce1).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   98.51%   98.55%   +0.03%     
==========================================
  Files          18       18              
  Lines        1009     1035      +26     
==========================================
+ Hits          994     1020      +26     
  Misses         15       15              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Collaborator

@Asanto32 Asanto32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few things to change, mainly to make sure it works with negtaive x/y changes

Comment thread src/graphomotor/features/trails/drawing_metrics.py
Comment thread src/graphomotor/features/trails/drawing_metrics.py
Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
Comment thread tests/unit/test_trails_drawing_metrics.py Outdated
Comment thread tests/unit/test_trails_drawing_metrics.py Outdated
Comment thread tests/unit/test_trails_drawing_metrics.py
@cgmaiorano cgmaiorano requested a review from Asanto32 January 19, 2026 18:44
Copy link
Copy Markdown
Collaborator

@Asanto32 Asanto32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks pretty good, makes more sense than first pass. Can we have negative values for x,y or is everything in the first quadrant?

Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
@cgmaiorano cgmaiorano requested a review from Asanto32 February 5, 2026 15:18
Copy link
Copy Markdown
Collaborator

@Asanto32 Asanto32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very minor comment about the name "arc_length". And whether this works with points when (x,y) < 0, ie.e outside the first quadrant (or is that impossible due to the Curious export?

Otherwise, letsgomets

Comment thread src/graphomotor/features/trails/drawing_metrics.py Outdated
@cgmaiorano
Copy link
Copy Markdown
Collaborator Author

Very minor comment about the name "arc_length". And whether this works with points when (x,y) < 0, ie.e outside the first quadrant (or is that impossible due to the Curious export?

Otherwise, letsgomets

I'm pretty sure negative points is impossible from curious export

@cgmaiorano cgmaiorano merged commit b310570 into main Feb 9, 2026
26 checks passed
@cgmaiorano cgmaiorano deleted the 59-task-write-trails-drawing-feature-functions-calculate_smoothness branch February 9, 2026 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Task: Write Trails Drawing feature functions: calculate_smoothness

2 participants